From: Chad Horohoe <demon@users.mediawiki.org>
Date: Sun, 5 Dec 2010 05:19:31 +0000 (+0000)
Subject: No need for globals, works fine without. Also add missing (and required) title arg
X-Git-Tag: 1.31.0-rc.0~33559
X-Git-Url: http://git.cyclocoop.org/%22.%24match%5B1%5D.%22?a=commitdiff_plain;h=132ef2968c4a26bf573449d4d576e1bdb5910bf2;p=lhc%2Fweb%2Fwiklou.git

No need for globals, works fine without. Also add missing (and required) title arg
---

diff --git a/maintenance/protect.php b/maintenance/protect.php
index 15780fbb5f..e97fc108b6 100644
--- a/maintenance/protect.php
+++ b/maintenance/protect.php
@@ -28,10 +28,11 @@ class Protect extends Maintenance {
 		$this->addOption( 'semiprotect', 'Adds semi-protection' );
 		$this->addOption( 'u', 'Username to protect with', false, true );
 		$this->addOption( 'r', 'Reason for un/protection', false, true );
+		$this->addArg( 'title', 'Title to protect', true );
 	}
 
 	public function execute() {
-		global $wgUser, $wgTitle, $wgArticle;
+		global $wgUser;
 
 		$userName = $this->getOption( 'u', 'Maintenance script' );
 		$reason = $this->getOption( 'r', '' );
@@ -46,16 +47,16 @@ class Protect extends Maintenance {
 		$wgUser = User::newFromName( $userName );
 		$restrictions = array( 'edit' => $protection, 'move' => $protection );
 
-		$wgTitle = Title::newFromText( $this->getArg() );
-		if ( !$wgTitle ) {
+		$t = Title::newFromText( $this->getArg() );
+		if ( !$t ) {
 			$this->error( "Invalid title", true );
 		}
 
-		$wgArticle = new Article( $wgTitle );
+		$article = new Article( $t );
 
 		# un/protect the article
 		$this->output( "Updating protection status... " );
-		$success = $wgArticle->updateRestrictions( $restrictions, $reason );
+		$success = $article->updateRestrictions( $restrictions, $reason );
 		if ( $success ) {
 			$this->output( "done\n" );
 		} else {